Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adsk contrib - Update builtin config and add studio builtin config #1713

Conversation

cedrik-fuoco-adsk
Copy link
Contributor

@cedrik-fuoco-adsk cedrik-fuoco-adsk commented Oct 27, 2022

Since the OCIO Configs Working Groups has released the stable version of the new OpenColorIO Configuration(s) for ACES, the builtin configs have to be updated.

The currently available CG config has been updated to the new version, and the Studio config has been added to the builtin configs registry.

Here are some examples of how they can be used with OCIO environment variable:

Use the default builtin config (cg-config-v0.1.0_aces-v1.3_ocio-v2.1.1) ---> ocio://default
Using the new CG config --> ocio://cg-config-v0.1.0_aces-v1.3_ocio-v2.1.1
Using the new Studio config -->ocio://studio-config-v1.0.0_aces-v1.3_ocio-v2.1

…I named the config without the RC2 in order to minimize the changes once the stable release in available.

Signed-off-by: Cedrik Fuoco <[email protected]>
Signed-off-by: Cedrik Fuoco <[email protected]>
- Added some variables for string const

Signed-off-by: Cedrik Fuoco <[email protected]>
@zachlewis
Copy link
Collaborator

Do we still want the "CG" config to be the default "ocio://default" config? Or did we intend to change that to the Studio config, now that a v1.0.0 has been released?

(I have no personal preference)

@doug-walker doug-walker merged commit cac1ba5 into AcademySoftwareFoundation:main Oct 31, 2022
@doug-walker doug-walker deleted the adsk_contrib/update-builtin-config-and-add-studio-builtin-config branch October 31, 2022 20:07
@doug-walker doug-walker mentioned this pull request Nov 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants